Skip to content

Conversation

Naveen-Gopu-F5
Copy link
Contributor

Proposed changes

  • Added Native OIDC implementation process in NginxaaS docs.

Checklist

Before sharing this pull request, I completed the following checklist:

Set up OIDC authentication _ DEV -- docs-nginx-com.pdf

image

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@Naveen-Gopu-F5 Naveen-Gopu-F5 requested a review from a team as a code owner September 26, 2025 07:53
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nginxaas-azure Issues related to NGINX as a Service for Azure labels Sep 26, 2025
Copy link
Contributor

@russokj russokj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conditionally approving assuming you address my concerns.

@Naveen-Gopu-F5 Naveen-Gopu-F5 force-pushed the NLB-7031-native-oidc-docs-for-nginxaas branch from cb6691e to d85818f Compare September 29, 2025 12:25
Copy link
Contributor

@JTorreG JTorreG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved with nits

@Naveen-Gopu-F5 Naveen-Gopu-F5 force-pushed the NLB-7031-native-oidc-docs-for-nginxaas branch 2 times, most recently from b2d76ce to 8295230 Compare October 7, 2025 18:25
@Naveen-Gopu-F5 Naveen-Gopu-F5 force-pushed the NLB-7031-native-oidc-docs-for-nginxaas branch from 8295230 to bc82390 Compare October 8, 2025 05:52
@JTorreG JTorreG merged commit 18200d3 into nginx:main Oct 8, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation product/nginxaas-azure Issues related to NGINX as a Service for Azure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants